Skip to content

Feat: added business form#86

Open
josephchimebuka wants to merge 1 commit intoPayStell:mainfrom
josephchimebuka:Business-Profile-Integration
Open

Feat: added business form#86
josephchimebuka wants to merge 1 commit intoPayStell:mainfrom
josephchimebuka:Business-Profile-Integration

Conversation

@josephchimebuka
Copy link
Copy Markdown

📌 Pull Request Description

📝 Summary

Added the business for merchants to update business profile

🔗 Related Issues

Closes #73

🔄 Changes Made

This PR introduces the Business Profile Management feature, allowing merchants to create, update, and manage their business profiles. The form includes fields for business name, contact email, business type, description, and logo upload. Changes include

🖼️ Current Output

Provide visual evidence of the changes:

  • For small changes: Screenshots.
Screenshot 2025-03-29 at 14 55 35
  • For large changes: Video or Loom link.

🧪 Testing

If applicable, describe the tests performed. Include screenshots, test outputs, or any resources that help reviewers understand how the changes were tested.

✅ Testing Checklist

  • Unit tests added/modified
  • Integration tests performed
  • Manual tests executed
  • All tests pass in CI/CD

⚠️ Potential Risks

List any possible issues that might arise with this change.

🚀 Next Steps & Improvements

This change lays a solid foundation for further optimizations. Some areas that could benefit from future improvements include:

  • 🔹 Performance optimization
  • 🔹 Increased test coverage
  • 🔹 Potential user experience enhancements

💬 Comments

Any additional context, questions, or considerations for reviewers.

@MPSxDev
Copy link
Copy Markdown
Contributor

MPSxDev commented Mar 31, 2025

This had already been created; the issue was with the BE implementation. There was no need to redo everything in the FE. @josephchimebuka

@josephchimebuka
Copy link
Copy Markdown
Author

Understood ser @MPSxDev I have added the code for the backend I thought the frontend needed another different input form for Business Profile

This had already been created; the issue was with the BE implementation. There was no need to redo everything in the FE. @josephchimebuka

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants